Skip to content

OpenSSF silver badge: reproducible build #1240

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
TonyXiang8787 merged 32 commits into from
Jan 13, 2026
Merged

OpenSSF silver badge: reproducible build #1240

TonyXiang8787 merged 32 commits into from
Jan 13, 2026

Conversation

@mgovers
Copy link
Member

@mgovers mgovers commented Dec 18, 2025 •
edited
Loading

Cherry-picked from #1228

This investigates the possibilities of producing a reproducible build

Before merge (do-not-merge label)

Relates to #1190

mgovers added 12 commits December 18, 2025 16:25
@mgovers
cmake reproducible builds
113d993 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
reproducible builds + workflow
0f432fb 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
add reproducible build to github actions
c916bf0 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
rename steps
23b76e9 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
gcc reproducibility
4914dc9 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
fix preset
f851552 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
fix typo
8f39a2e 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
fix typo
db86996 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
fix correct gcc version
b0b4094 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
fix msvc conda prefix
374754d 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
clean up build dir
8ca8a35 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
remove unused
623452f 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers mgovers self-assigned this Dec 18, 2025
@mgovers mgovers added do-not-merge This should not be merged improvement Improvement on internal implementation labels Dec 18, 2025
mgovers
mgovers commented Dec 18, 2025
View reviewed changes
mgovers and others added 8 commits December 18, 2025 16:46
mgovers
mgovers commented Jan 12, 2026
View reviewed changes
@mgovers
reproducible build windows
12a7fc4 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
Copy link
Member Author

mgovers commented Jan 12, 2026

Reproducible build on windows successful with both msvc and clang-cl in https://github.com/PowerGridModel/power-grid-model/actions/runs/20923965759/job/60116675758?pr=1240 (12a7fc4)

@mgovers
fix script for linux
09d6416 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
test script for macos
1da6be5 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
fix typo
ae9e722 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers mgovers marked this pull request as ready for review January 13, 2026 10:35
@mgovers mgovers removed the do-not-merge This should not be merged label Jan 13, 2026
@mgovers
Copy link
Member Author

mgovers commented Jan 13, 2026

do-not-merge label removed because the reproducible build check passes in the nightly run https://github.com/PowerGridModel/power-grid-model/actions/runs/20953474927

TonyXiang8787
TonyXiang8787 reviewed Jan 13, 2026
View reviewed changes
TonyXiang8787
TonyXiang8787 reviewed Jan 13, 2026
View reviewed changes
@mgovers
resolve some (not all) comments
c7156f8 
Signed-off-by: Martijn Govers <[email protected]>
TonyXiang8787
TonyXiang8787 reviewed Jan 13, 2026
View reviewed changes
@mgovers
Copy link
Member Author

mgovers commented Jan 13, 2026 •
edited
Loading

Restarted Nightly in https://github.com/PowerGridModel/power-grid-model/actions/runs/20956375532

TonyXiang8787
TonyXiang8787 previously approved these changes Jan 13, 2026
View reviewed changes
@mgovers
Copy link
Member Author

mgovers commented Jan 13, 2026

Restarted Nightly in https://github.com/PowerGridModel/power-grid-model/actions/runs/20956375532

this nightly failed on the macos one. Investigating now.

@mgovers
apple clang preset
aafc5ae 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
Copy link
Member Author

mgovers commented Jan 13, 2026 •
edited
Loading

Restarted Nightly in https://github.com/PowerGridModel/power-grid-model/actions/runs/20956375532

this nightly failed on the macos one. Investigating now.

fix running in https://github.com/PowerGridModel/power-grid-model/actions/runs/20962280036

@mgovers
add forgotten build preset
7c55026 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
cmake shared linker flags
df67e5e 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
reverse + reproducibility
a75da94 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
typo
59a654d 
Signed-off-by: Martijn Govers <[email protected]>
@mgovers
Copy link
Member Author

mgovers commented Jan 13, 2026

another one running in https://github.com/PowerGridModel/power-grid-model/actions/runs/20963875918

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 13, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@TonyXiang8787 TonyXiang8787 added this pull request to the merge queue Jan 13, 2026
Merged via the queue into main with commit e7b073d Jan 13, 2026
59 checks passed
@TonyXiang8787 TonyXiang8787 deleted the feature/reproducible-build branch January 13, 2026 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TonyXiang8787 TonyXiang8787 TonyXiang8787 approved these changes

Assignees

@mgovers mgovers

Labels

improvement Improvement on internal implementation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mgovers @TonyXiang8787